Skip to content

Import missing ExprTrait#2606

Merged
tyt2y3 merged 10 commits intoSeaQL:masterfrom
Huliiiiii:remove-inherent
Jun 9, 2025
Merged

Import missing ExprTrait#2606
tyt2y3 merged 10 commits intoSeaQL:masterfrom
Huliiiiii:remove-inherent

Conversation

@Huliiiiii
Copy link
Copy Markdown
Member

PR Info

  • Closes
  • Dependents:

New Features

Bug Fixes

Breaking Changes

Changes

Huliiiiii added 3 commits May 23, 2025 09:37
- Call `cast_as` in fully qualified syntax in `DeriveEntityModel`
@Huliiiiii
Copy link
Copy Markdown
Member Author

I found a issue where ExprTrait::as_enum would conflict with ActiveEnum::as_enum

Copy link
Copy Markdown
Member

@Expurple Expurple left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you. Everything looks good, but cargo test --workspace fails. You need to add two more imports.

After that, we can trigger a CI run to check other test configurations

Copy link
Copy Markdown
Member

@Expurple Expurple left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tyt2y3, can you trigger a CI run? The PR seems ready in general

@Huliiiiii Huliiiiii marked this pull request as ready for review May 30, 2025 12:53
Copy link
Copy Markdown
Member

@Expurple Expurple left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything seems good in both PRs, thank you.

Now we can:

  • merge SeaQL/sea-query#890
  • then rollback the Cargo.toml changes here (should've kept the Draft status until then, so that we don't forget)
  • merge this PR

@Huliiiiii Huliiiiii marked this pull request as draft June 1, 2025 12:04
@Expurple
Copy link
Copy Markdown
Member

Expurple commented Jun 8, 2025

SeaQL/sea-query#890 has been merged.

Now we can rollback the Cargo.toml changes and merge this

@Expurple
Copy link
Copy Markdown
Member

Expurple commented Jun 9, 2025

@tyt2y3, you can merge this now.

The new CI failures come from SeaQL/sea-query#891 (#2607 not being merged yet)

@Expurple Expurple marked this pull request as ready for review June 9, 2025 05:56
@tyt2y3 tyt2y3 merged commit c486695 into SeaQL:master Jun 9, 2025
9 of 33 checks passed
@Huliiiiii Huliiiiii deleted the remove-inherent branch October 9, 2025 20:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants